Reviewing files that changed from the base of the PR and between b42b428 and 09b338b.

• components/sharepoint/actions/create-folder/create-folder.mjs (1 hunks)
• components/sharepoint/actions/create-item/create-item.mjs (1 hunks)
• components/sharepoint/actions/create-link/create-link.mjs (1 hunks)
• components/sharepoint/actions/create-list/create-list.mjs (1 hunks)
• components/sharepoint/actions/download-file/download-file.mjs (1 hunks)
• components/sharepoint/actions/find-file-by-name/find-file-by-name.mjs (1 hunks)
• components/sharepoint/actions/get-excel-table/get-excel-table.mjs (1 hunks)
• components/sharepoint/actions/get-file-by-id/get-file-by-id.mjs (1 hunks)
• components/sharepoint/actions/list-files-in-folder/list-files-in-folder.mjs (1 hunks)
• components/sharepoint/actions/update-item/update-item.mjs (1 hunks)
• components/sharepoint/actions/upload-file/upload-file.mjs (1 hunks)
• components/sharepoint/common/constants.mjs (1 hunks)
• components/sharepoint/package.json (1 hunks)
• components/sharepoint/sharepoint.app.mjs (4 hunks)
• components/sharepoint/sources/common/common.mjs (2 hunks)
• components/sharepoint/sources/new-file-created/new-file-created.mjs (1 hunks)
• components/sharepoint/sources/new-folder-created/new-folder-created.mjs (1 hunks)
• components/sharepoint/sources/new-list-item/new-list-item.mjs (1 hunks)
• components/sharepoint/sources/updated-list-item/updated-list-item.mjs (1 hunks)

• GitHub Check: pnpm publish
• GitHub Check: Lint Code Base
• GitHub Check: Verify TypeScript components
• GitHub Check: Publish TypeScript components

components/sharepoint/actions/download-file/download-file.mjs (1)

8-8: LGTM! Routine version bump.

Version bump aligns with the broader SharePoint component expansion in this PR.

components/sharepoint/actions/create-list/create-list.mjs (1)

7-7: LGTM! Routine version bump.

Version increment is consistent with the PR's updates to the SharePoint integration.

components/sharepoint/package.json (1)

3-3: LGTM! Appropriate minor version bump.

The bump from 0.3.3 to 0.4.0 correctly signals the addition of new SharePoint actions and sources in this PR.

components/sharepoint/actions/update-item/update-item.mjs (1)

8-8: LGTM! Routine version bump.

Version increment aligns with the PR's component updates.

components/sharepoint/sources/updated-list-item/updated-list-item.mjs (1)

7-8: LGTM! Clarified description.

The description now correctly indicates the source emits events for list item updates rather than list updates, improving clarity.

components/sharepoint/sources/common/common.mjs (2)

26-31: LGTM! Extension points for derived sources.

These methods provide customization hooks for sources to control sorting order and item relevance filtering.

---

56-69: LGTM! Enhanced emission logic with first-run protection.

The changes add:

• First-run protection: limits to 10 items when lastTs is falsy
• Filtering: respects isRelevant() for item selection
• Early exit: breaks when items are sorted descending and timestamp threshold is reached

This prevents overwhelming users with historical data on initial runs while maintaining flexibility for derived sources.

components/sharepoint/sources/new-list-item/new-list-item.mjs (1)

7-8: LGTM! Clarified description.

The description now correctly indicates the source emits events for new list items rather than new lists, improving accuracy and consistency with the updated-list-item source.

components/sharepoint/actions/create-item/create-item.mjs (1)

7-7: Version bump noted

No issues with the metadata change.

components/sharepoint/actions/find-file-by-name/find-file-by-name.mjs (1)

35-42: Guard empty name, add select params to trim payload; confirm single-page results are acceptable.

searchDriveItems does not auto-paginate—it returns a single response. For "find-file-by-name," verify that single-page results are sufficient or implement pagination handling if needed.

Apply:

   async run({ $ }) {
+    const query = (this.name || "").trim();
+    if (!query) throw new Error("File Name is required.");
     const response = await this.sharepoint.searchDriveItems({
       $,
       siteId: this.siteId,
-      query: this.name,
+      query,
+      params: { select: "id,name,folder" },
     });
     let values = response.value.filter(
-      ({ name }) => name.toLowerCase().includes(this.name.toLowerCase()),
+      ({ name }) => name?.toLowerCase().includes(query.toLowerCase()),
     );

components/sharepoint/actions/create-link/create-link.mjs (1)

57-70: Add validation guard for unsupported embed type; include URL in summary.

The embed sharing link type is only supported on personal OneDrive files per the Microsoft Graph API. This SharePoint action uses siteId (not personal OneDrive), so embed will fail at the API level. Adding early validation improves UX. Additionally, the API response includes link.webUrl, which should be exported in the summary.

Note: driveId is used for dynamic filtering but is not required by createLink()—the method uses the site-based path /sites/{siteId}/drive/items/{fileId}/createLink.

   async run({ $ }) {
+    if (this.type === "embed") {
+      throw new Error("Embed links are only supported for personal OneDrive files.");
+    }
     const response = await this.sharepoint.createLink({
       $,
       siteId: this.siteId,
       fileId: this.fileId,
       data: {
         type: this.type,
         scope: this.scope,
       },
     });
 
     if (response?.id) {
-      $.export("$summary", `Successfully created a sharing link with ID \`${response.id}\`.`);
+      const url = response?.link?.webUrl;
+      $.export("$summary", `Created sharing link ${response.id}${url ? ` → ${url}` : ""}.`);
     }

components/sharepoint/sources/new-file-created/new-file-created.mjs (3)

3-10: LGTM! Metadata is well-defined.

The component metadata follows standard patterns with appropriate dedupe strategy and clear naming.

---

11-38: LGTM! Props are correctly configured.

The prop definitions properly chain dependencies (driveId → siteId, folderId → siteId + driveId) and reference the SharePoint app propDefinitions correctly.

---

62-63: LGTM! File filtering logic is correct.

The isRelevant method correctly filters to only items with a file property, excluding folders.